Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Backport] Minor CI improvements #400

Open
wants to merge 2 commits into
base: 11.8.x
Choose a base branch
from
Open

[Backport] Minor CI improvements #400

wants to merge 2 commits into from
+4 −4

Conversation

leofang
Copy link
Member

@leofang leofang commented Jan 15, 2025

Backport the relevant bits from #393.

@leofang leofang added P1 Medium priority - Should do CI/CD CI/CD infrastructure labels Jan 15, 2025
@leofang leofang added this to the cuda-python 12-next, 11-next milestone Jan 15, 2025
@leofang leofang requested a review from rwgk January 15, 2025 23:06
@leofang leofang self-assigned this Jan 15, 2025
@copy-pr-bot copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Jan 15, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@leofang
Copy link
Member Author

leofang commented Jan 15, 2025

@shwina do you know if we we could disable pre-commit CI for backport PRs, or should we just backport a dummy config file?

rwgk
rwgk previously approved these changes Jan 16, 2025
View reviewed changes
@leofang
Copy link
Member Author

leofang commented Jan 16, 2025

/ok to test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rwgk rwgk rwgk left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@leofang leofang

Labels
CI/CD CI/CD infrastructure P1 Medium priority - Should do
Projects
None yet
Milestone
cuda-python 12-next, 11-next
Development

Successfully merging this pull request may close these issues.
2 participants
@leofang @rwgk